-
Notifications
You must be signed in to change notification settings - Fork 9
Support authorizing asana requests via tokens #130
Conversation
159a5c5 to
5fbde43
Compare
| class CC::Service::Asana < CC::Service | ||
| class Config < CC::Service::Config | ||
| attribute :api_key, Axiom::Types::String, label: "API key" | ||
| attribute :personal_access_token, Axiom::Types::String, label: "Personal Access Token" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we treat this like the GH tokens and only display the full thing once?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, good call. I'll open an app PR that uses this branch and adds the obfuscation there
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool - wasn't sure if we needed to do something special here as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought we would need to do something special, but while testing locally it seems like everything Just Works ™️ even without changing the views in app :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Even better!
gdiggs
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
dd881d1 to
8d1da1f
Compare
Asana is deprecating their API Key access, as reported by a helpful user. https://asana.com/developers/documentation/getting-started/auth#api-key We confirmed with their support team that they're planning to drop support for API Key authorization in "early 2017". This PR introduces support for authorizing with personal access tokens, and attempts to nudge users in that direction gently. I think we'll need to make a plan about how to migrate existing repos over.
8d1da1f to
af83a9a
Compare
Asana is deprecating their API Key access, as reported by a helpful
user.
https://asana.com/developers/documentation/getting-started/auth#api-key
We confirmed with their support team that they're planning to drop
support for API Key authorization in "early 2017".
This PR introduces support for authorizing with personal access tokens,
and attempts to nudge users in that direction gently. I think we'll need
to make a plan about how to migrate existing repos over.